-
-
Notifications
You must be signed in to change notification settings - Fork 359
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update cinnabar style to 7.0.1 #323
update cinnabar style to 7.0.1 #323
Conversation
fixes part of streetcomplete#199
type: TopoJSON #GeoJSON, MVT, TopoJSON | ||
url: https://tile.mapzen.com/mapzen/vector/v1/all/{z}/{x}/{y}.topojson | ||
#url: '//tile.dev.mapzen.com/mapzen/vector/v1/all/{z}/{x}/{y}.topojson' | ||
#url: //localhost:8080/vector/all/{z}/{x}/{y}.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This must stay MVT!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting. I missed it, but I tested it and it works nicely (at my phone at least)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it accepts any of these formats. But the binary MVT is smaller, this makes a difference with this many users.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Switched back to MVT, it should be now ready.
cool, thanks |
f4abb97#commitcomment-22538246 Could you please change the url to the mvt url as well and use http (not https)? |
done and documented in #328 |
fixes part of #199 (display barriers that are important during survey)